Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Turbo Blinds Documentation Tests #711

Closed
wants to merge 7 commits into from

Conversation

Anunayj
Copy link
Contributor

@Anunayj Anunayj commented Apr 12, 2022

Turbo Blind (Blind Lending protocol) documentation, Test is done for most part, but needs (a lot of) review, any contributions are welcome.

Protocol Summary here: https://gist.github.com/Anunayj/750c85c9bea6a707e7e32a72f9ef6e83

Ignore any changes except for test/turbo-blinds-test.js for now, depends on #666, will rebase once that is merged.

Anunayj and others added 6 commits October 2, 2021 21:08
MTX clone() would not carry over coinview, in order to get around this the CoinSelector
would remove all inputs and add them back which made working with partially signed
transactions a pain, this should fix that issue.
Previous version of interactive-name-swap used a ugly hack to get around limitations
on wallet funding, since wallet.fund() no longer wipes previous inputs, the previous
implementation is not longer the recommended way.
@Anunayj Anunayj changed the title Turbo Blinds Documentation Tests test: Turbo Blinds Documentation Tests Apr 12, 2022
@Anunayj Anunayj added waiting for other PR modifier needs rebase modifier needs conceptual review modifier docs part of the codebase tests part of the codebase labels Apr 12, 2022
@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 64.637% when pulling 2f4bd3b on Anunayj:staking into 35ea46d on handshake-org:master.

@Anunayj Anunayj marked this pull request as ready for review April 13, 2022 03:26
@Anunayj Anunayj marked this pull request as draft April 13, 2022 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs part of the codebase needs conceptual review modifier needs rebase modifier tests part of the codebase waiting for other PR modifier
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants